Skip to content

[FIX] 스웨거 막기 광야의 것 탐 내지마라#195

Merged
PBEM22 merged 1 commit intodevelopfrom
fix/#193/swagger
Jan 23, 2026
Merged

[FIX] 스웨거 막기 광야의 것 탐 내지마라#195
PBEM22 merged 1 commit intodevelopfrom
fix/#193/swagger

Conversation

@PBEM22
Copy link
Copy Markdown
Collaborator

@PBEM22 PBEM22 commented Jan 23, 2026 •
edited by coderabbitai bot
Loading

📌 관련 이슈

✨ 변경 사항

  • 스웨거 막기 광야의 것 탐 내지마라

📸 테스트 증명 (필수)

📚 리뷰어 참고 사항

✅ 체크리스트

  • 브랜치 전략(git flow)을 따랐나요? (develop -> feat/...)
  • 로컬에서 빌드 및 실행이 정상적으로 되나요?
  • 불필요한 주석(TODO 등)이나 더미 코드는 제거했나요?
  • 컨벤션(커밋 메시지, 코드 스타일)을 지켰나요?

Summary by CodeRabbit

릴리스 노트

  • 설정 변경
    • API 문서 및 Swagger UI 접근 경로가 기본값으로 변경되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@PBEM22 PBEM22 self-assigned this Jan 23, 2026
@PBEM22 PBEM22 added 🚨 Fix 버그 수정 🔥 P0 당장 개발해야 하는 핵심 기능 labels Jan 23, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 23, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

application.yml에서 springdoc API 문서와 Swagger UI의 경로 설정을 제거했습니다. 이제 라이브러리의 기본 설정값을 사용하게 됩니다.

Changes

Cohort / File(s) 요약
스웨거 설정 제거
src/main/resources/application.yml		 springdoc.api-docs.path와 springdoc.swagger-ui.path 설정 제거 (기본값으로 변경)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3분

🚥 Pre-merge checks | ✅ 3 | ❌ 2
❌ Failed checks (2 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive PR 제목이 변경 사항과 관련되어 있지만, 구체적이지 않고 모호한 표현으로 인해 실제 변경 내용을 명확히 파악하기 어렵습니다. 제목을 더 구체적으로 수정하세요. 예: '[FIX] Swagger API docs/UI 경로 설정 제거' 같이 변경 사항을 명확히 드러내는 표현 권장.
Linked Issues check ❓ Inconclusive 이슈 #193의 요구사항이 모호해서('광야의 것 탐내지마라') 코드 변경이 실제로 요구사항을 충족하는지 명확히 확인할 수 없습니다. 이슈 #193의 상세 요구사항을 명확히 정의하세요. Swagger 차단의 목적(전체 비활성화? 특정 경로만?)과 예상 결과를 구체적으로 기술해야 합니다.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Out of Scope Changes check ✅ Passed application.yml의 springdoc 설정 경로 제거가 Swagger 차단과 직접적으로 연관되어 있어 범위 내 변경으로 보입니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]
coderabbitai bot reviewed Jan 23, 2026
View reviewed changes
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/main/resources/application.yml (1)

98-106: 스웨거가 실제로 차단되지 않았습니다.

현재 변경 사항으로는 스웨거가 차단되지 않고 기본 경로로 여전히 접근 가능합니다. path 설정만 제거했을 뿐 enabled: true가 유지되고 있어서, 스웨거는 다음 기본 경로에서 계속 동작합니다:

  • API 문서: /v3/api-docs
  • Swagger UI: /swagger-ui.html 또는 /swagger-ui/index.html

PR 목적(#193: 스웨거 막기)을 달성하려면 아래와 같이 수정해야 합니다.

🔒 스웨거를 완전히 비활성화하는 방법 
 springdoc:
   api-docs:
-    enabled: true
+    enabled: false
   swagger-ui:
-    enabled: true
+    enabled: false
-    groups-order: DESC   # 그룹 정렬 순서
-    operations-sorter: method # 메소드 순서대로 정렬
-    disable-swagger-default-url: true # 기본 petstore URL 끄기
-    display-request-duration: true # 요청 소요 시간 표시

또는 프로파일별로 제어하고 싶다면 프로덕션 환경에서만 비활성화할 수도 있습니다.

@PBEM22 PBEM22 merged commit cc649e9 into develop Jan 23, 2026
2 checks passed
@PBEM22 PBEM22 deleted the fix/#193/swagger branch January 23, 2026 09:45
@88guri
Copy link
Copy Markdown
Collaborator

88guri commented Jan 23, 2026

포티 스웨거 탐내지 마라 ㅡ , ㅡ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Assignees

@PBEM22 PBEM22

Labels

🚨 Fix 버그 수정 🔥 P0 당장 개발해야 하는 핵심 기능

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FIX] 스웨거 막기

2 participants

@PBEM22 @88guri